fix(build): recover ConfigDumpInfo after failed Designer build - #47

src/use_cases/build_project/cdfi_recovery.rs (1)

108-149: 🩺 Stability & Availability | 🔵 Trivial | 💤 Low value

Повторный вызов restore() после успешного восстановления вернёт ошибку.

После finalize_successful_restore snapshot удалён, поэтому повторный restore() (если guard когда-нибудь переиспользуют) упадёт в restore_snapshot_with на чтении снимка вместо возврата NotNeeded. Сейчас вызывающий код делает это один раз, но защита от будущего рефакторинга дешёвая: помечать guard как «уже восстановлен» и сразу отдавать NotNeeded.

🤖 Prompt for AI Agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@src/use_cases/build_project/cdfi_recovery.rs` around lines 108 - 149,
Добавьте в состояние guard признак успешного восстановления и обновляйте его
после завершения restore; в restore_with проверяйте этот признак до обращения к
snapshot и возвращайте CdfiRecoverySummary с действием NotNeeded. Используйте
существующие значения tracked_path, original_exists и changed_entry_count, чтобы
повторный вызов restore() не пытался читать уже удалённый snapshot.

src/use_cases/build_project.rs (1)

632-645: 🗄️ Data Integrity & Integration | 🔵 Trivial | 🏗️ Heavy lift

Диагностика восстановления хранится только для одного source-set.

BuildResult.cdfi_recovery — один слот, а retain_cdfi_recovery/merge_cdfi_recovery в координаторе перезаписывают предыдущее значение (тест execute_build_keeps_designer_cdfi_replacement_after_successful_update явно ожидает summary для ext, а не для main). При сборке нескольких source-set сведения о предыдущих CDFI теряются, кроме склеенного cleanup-предупреждения. Если это осознанный компромисс — ок; иначе стоит перейти на Vec<CdfiRecoverySummary> в контракте.

🤖 Prompt for AI Agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@src/use_cases/build_project.rs` around lines 632 - 645, Измените контракт
результата сборки вокруг BuildStepOutcome и BuildResult::cdfi_recovery с одного
Option<CdfiRecoverySummary> на Vec<CdfiRecoverySummary>, чтобы сохранять
диагностику для каждого source-set. Обновите retain_cdfi_recovery и
merge_cdfi_recovery в координаторе для добавления новых сведений вместо
перезаписи предыдущих, сохранив существующее объединение cleanup-предупреждений.

src/use_cases/build_project/coordinator.rs (1)

986-999: 📐 Maintainability & Code Quality | 🔵 Trivial | ⚡ Quick win

retain_cdfi_recovery теряет второе cleanup-предупреждение.

Если у текущего summary уже есть cleanup_warning, кандидат отбрасывается целиком — включая его собственный cleanup_warning и удержанный snapshot_path. merge_cdfi_recovery в такой ситуации предупреждения склеивает; логично переиспользовать её и на успешном пути.

♻️ Возможный вариант

 fn retain_cdfi_recovery(
     current: &mut Option<Box<CdfiRecoverySummary>>,
     candidate: Option<Box<CdfiRecoverySummary>>,
 ) {
     let Some(candidate) = candidate else {
         return;
     };
-    if current
-        .as_ref()
-        .is_none_or(|summary| summary.cleanup_warning.is_none())
-    {
-        *current = Some(candidate);
-    }
+    *current = merge_cdfi_recovery(current.take(), Some(candidate));
 }

🤖 Prompt for AI Agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@src/use_cases/build_project/coordinator.rs` around lines 986 - 999, Update
retain_cdfi_recovery so a candidate is not discarded when the current summary
already has cleanup_warning; merge both summaries through merge_cdfi_recovery,
preserving the candidate’s cleanup warning and snapshot_path. Keep the existing
replacement behavior when current is absent or has no cleanup warning, and
retain the early return for a missing candidate.

docs/superpowers/plans/2026-07-26-cdfi-rollback.md (1)

56-59: 📐 Maintainability & Code Quality | 🔵 Trivial | 💤 Low value

\n в git commit -m не превратится в перевод строки.

В двойных кавычках bash оставит \n буквально; используйте $'...' или несколько -m. То же в шагах Task 2 и Task 3.

🤖 Prompt for AI Agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@docs/superpowers/plans/2026-07-26-cdfi-rollback.md` around lines 56 - 59,
Исправьте команды git commit в текущем шаге и шагах Task 2 и Task 3:
последовательность \n внутри двойных кавычек сейчас сохраняется буквально, а не
становится переносом строки. Используйте несколько аргументов -m либо ANSI-C
quoting через $'...', сохранив заданное содержимое сообщения коммита.

Fix all unresolved CodeRabbit comments on this PR:

• Push a commit to this branch (recommended)
• Create a new PR with the fixes